Skip to content

[#8937] fix(core): Fix flaky test in testPolicyAndTagCacheWeigher#8931

Merged
jerryshao merged 3 commits intoapache:mainfrom
yuqi1129:fix_flaky_test
Oct 29, 2025
Merged

[#8937] fix(core): Fix flaky test in testPolicyAndTagCacheWeigher#8931
jerryshao merged 3 commits intoapache:mainfrom
yuqi1129:fix_flaky_test

Conversation

@yuqi1129
Copy link
Copy Markdown
Contributor

@yuqi1129 yuqi1129 commented Oct 28, 2025

What changes were proposed in this pull request?

Comment out the flaky test cases.

Why are the changes needed?

As the weight mechanism in the Caffeine cache is probabilistic, the CI pipeline will fail occasionally(failure rate is about 10% or so).

Fix: #8937

Does this PR introduce any user-facing change?

N/A.

How was this patch tested?

The existing tests.

@yuqi1129 yuqi1129 closed this Oct 28, 2025
@yuqi1129 yuqi1129 reopened this Oct 28, 2025
@yuqi1129 yuqi1129 closed this Oct 28, 2025
@yuqi1129 yuqi1129 reopened this Oct 28, 2025
@yuqi1129 yuqi1129 closed this Oct 28, 2025
@yuqi1129 yuqi1129 reopened this Oct 28, 2025
@yuqi1129 yuqi1129 added the branch-1.0 Automatically cherry-pick commit to branch-1.0 label Oct 28, 2025
@yuqi1129 yuqi1129 changed the title Fix test [#8937] fix(core): Fix flaky test in testPolicyAndTagCacheWeigher Oct 28, 2025
@yuqi1129 yuqi1129 marked this pull request as ready for review October 28, 2025 11:47
@yuqi1129 yuqi1129 self-assigned this Oct 29, 2025
@yuqi1129 yuqi1129 requested review from jerryshao and mchades October 29, 2025 08:16
@jerryshao jerryshao merged commit 3b9b475 into apache:main Oct 29, 2025
26 checks passed
github-actions Bot pushed a commit that referenced this pull request Oct 29, 2025
)

### What changes were proposed in this pull request?

Comment out the flaky test cases.

### Why are the changes needed?

As the weight mechanism in the Caffeine cache is probabilistic, the CI
pipeline will fail occasionally(failure rate is about 10% or so).

Fix: #8937

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

The existing tests.
yuqi1129 added a commit to yuqi1129/gravitino that referenced this pull request Nov 3, 2025
…er (apache#8931)

### What changes were proposed in this pull request?

Comment out the flaky test cases.

### Why are the changes needed?

As the weight mechanism in the Caffeine cache is probabilistic, the CI
pipeline will fail occasionally(failure rate is about 10% or so).

Fix: apache#8937

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

The existing tests.
jerryshao pushed a commit that referenced this pull request Nov 4, 2025
)

### What changes were proposed in this pull request?

Comment out the flaky test cases.

### Why are the changes needed?

As the weight mechanism in the Caffeine cache is probabilistic, the CI
pipeline will fail occasionally(failure rate is about 10% or so).

Fix: #8937

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

The existing tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch-1.0 Automatically cherry-pick commit to branch-1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug report] Flaky test in TestCacheConfig#testPolicyAndTagCacheWeigher

2 participants